Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Grasshopper_Toolkit - Issue 226 - Being able to input arrays #389

Closed

Conversation

epignatelli
Copy link
Member

@epignatelli epignatelli commented Aug 1, 2019

NOTE: Depends on BHoM/BHoM_UI#117

Issues addressed by this PR

Closes #226

Test files

Note that, even you need to use BHoM/BHoM_Engine#1124 to be able to input arrays, although it does not represent a code-wise dependency.
https://burohappold.sharepoint.com/:f:/s/BHoM/EjmxIfvj4WNBvi9qWzVvunoBwjYJX03lohJTTu8WHAxAhA?e=JpiJCu

Changelog

  • Adds DataAccessor_GH.GetDataArray(int index)

Additional comments

This pr is to support arrays (e.g. int[]) as inputs from the visual programming interface. it does so by adding another getter GetDataArray(int).

There are a series of cases that this does not solve voluntarily, where the arrays are nested, e.g. List<int[]>

@epignatelli epignatelli changed the title Solves #226 Grasshopper_Toolkit - Issue 226 - Being able to input arrays Aug 1, 2019
@epignatelli epignatelli added severity:critical No workaround exists. Essential to continue type:feature New capability or enhancement labels Aug 1, 2019
@epignatelli epignatelli self-assigned this Aug 1, 2019
@epignatelli epignatelli added this to the BHoM 2.4 β MVP milestone Aug 1, 2019
@epignatelli epignatelli marked this pull request as ready for review August 1, 2019 14:26
@adecler
Copy link
Member

adecler commented Aug 3, 2019

See BHoM/BHoM_UI#117 for comment

@adecler adecler added the status:do-not-merge For instance, test PR, for discussion, or dependant PRs not ready for merge label Aug 3, 2019
@epignatelli
Copy link
Member Author

epignatelli commented Aug 8, 2019

Closing this because of BHoM/BHoM_UI#117 (comment)

@epignatelli epignatelli closed this Aug 8, 2019
@epignatelli epignatelli deleted the Grasshopper_Toolkit-Issue226-InputsArrays branch September 11, 2019 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity:critical No workaround exists. Essential to continue status:do-not-merge For instance, test PR, for discussion, or dependant PRs not ready for merge type:feature New capability or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Grasshopper_Toolkit: Cannot input an array of objects
2 participants